Replace dependency passport-saml with @node-saml/passport-saml ^4.0.4#12489
Replace dependency passport-saml with @node-saml/passport-saml ^4.0.4#12489renovate[bot] wants to merge 1 commit intomasterfrom
Conversation
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
🤖 Augment PR SummarySummary: This PR replaces the SAML Passport strategy dependency used by the auth-backend plugin. Changes:
Why: Renovate-generated “replacement” update to move off the legacy Technical Notes: Any runtime/TS imports that still reference 🤖 Was this summary useful? React with 👍 or 👎 |
| "passport-oauth2": "^1.6.1", | ||
| "passport-onelogin-oauth": "^0.0.1", | ||
| "passport-saml": "^3.1.2", | ||
| "@node-saml/passport-saml": "^4.0.4", |
There was a problem hiding this comment.
plugins/auth-backend/src/providers/saml/provider.ts still imports from passport-saml (and passport-saml/lib/...), so switching the dependency to @node-saml/passport-saml will likely cause module resolution failures at build/runtime unless those import paths are updated too.
Severity: high
Other Locations
plugins/auth-backend/src/providers/saml/provider.ts:18plugins/auth-backend/src/providers/saml/provider.ts:23
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
EntelligenceAI PR SummaryMigrates SAML authentication library from deprecated
Confidence Score: 4/5 - Mostly SafeLikely safe but review recommended — this PR correctly migrates Key Findings:
Files requiring special attention
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
This PR contains the following updates:
^3.1.2→^4.0.4This is a special PR that replaces
passport-samlwith the community suggested minimal stable replacement version.Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.